Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add how to use EFA to faq.rst #3818

Closed
wants to merge 2 commits into from

Conversation

jasonkrone
Copy link

Added instructions for how to use elastic fiber adapter (EFA) with a capacity reservation on AWS to faq.rst.

Tested (run the relevant ones):

  • Code formatting: bash format.sh
  • Any manual or new tests for this PR (please specify below)
  • All smoke tests: pytest tests/test_smoke.py
  • Relevant individual smoke tests: pytest tests/test_smoke.py::test_fill_in_the_name
  • Backward compatibility tests: conda deactivate; bash -i tests/backward_compatibility_tests.sh

Regarding tests, I only confirmed via a local .rst render tool that the text was formatting correctly. Note: for some reason the text for items number 5 and 6 in my FAQ answer are italicized and I couldn't figure out how to fix it.

@zaptrem
Copy link

zaptrem commented Oct 23, 2024

This is crazy complicated. Is there a simpler/more intended way of accomplishing this?

@Michaelvll
Copy link
Collaborator

This is crazy complicated. Is there a simpler/more intended way of accomplishing this?

Thank you for your interest @zaptrem! Would you like to try out this? #4062

Copy link
Contributor

This PR is stale because it has been open 120 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Feb 21, 2025
Copy link
Contributor

github-actions bot commented Mar 3, 2025

This PR was closed because it has been stalled for 10 days with no activity.

@github-actions github-actions bot closed this Mar 3, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants